NEWS: update

babl: drop a now deprecated const

babl_format_new: also free doc to plug leak

doc and name are allocated via babl_strdup, hence if we don't use them
we need to free both of them - therefore add a free(doc) to match the
existing free(name).

format_new (called just below) transfers ownership of doc, hence it's
correct not to free doc in the case where we're instantiating a new
format - whereas name is copied within format_new, and therefore needs to be
free'd in either scenario.

Leak seen while running gimp+babl built with ASAN:

  Direct leak of 99 byte(s) in 1 object(s) allocated from:
    #0 0x5e87cd in malloc /home/abuild/rpmbuild/BUILD/llvm-12.0.0.src/build/../projects/compiler-rt/lib/asan/asan_malloc_linux.cpp:145:3
    #1 0x7f038605e2b8 in babl_malloc /home/ahunt/git/babl/_build/../babl/babl-memory.c:115:9
    #2 0x7f038605eec7 in babl_strdup /home/ahunt/git/babl/_build/../babl/babl-memory.c:296:9
    #3 0x7f038605110f in babl_format_new /home/ahunt/git/babl/_build/../babl/babl-format.c:376:17
    #4 0x7f037313a432 in init /home/ahunt/git/babl/_build/../extensions/cairo.c:569:25
    #5 0x7f03860412a1 in babl_extension_load /home/ahunt/git/babl/_build/../babl/babl-extension.c:226:7
    #6 0x7f0386040d1c in babl_extension_load_dir /home/ahunt/git/babl/_build/../babl/babl-extension.c:268:19
    #7 0x7f03860409a8 in babl_extension_load_dir_list /home/ahunt/git/babl/_build/../babl/babl-extension.c:338:17
    #8 0x7f038607d6cd in babl_init /home/ahunt/git/babl/_build/../babl/babl.c:154:7
    #9 0x7f0386bce5ab in gegl_post_parse_hook /home/ahunt/git/gegl/_build/../gegl/gegl-init.c:542:3
    #10 0x7f03844aad8a in g_option_context_parse /home/ahunt/git/glib/_build/../glib/goption.c:2228:12
    #11 0x7f0386bce381 in gegl_init /home/ahunt/git/gegl/_build/../gegl/gegl-init.c:189:8
    #12 0xef1f6b in gimp_init_for_testing /home/ahunt/git/gimp/app/tests.c:69:3
    [...]

SUMMARY: AddressSanitizer: 99 byte(s) leaked in 1 allocation(s).

Add CIE Lab alpha u16 format

Add CIE Lab alpha u8 format

babl: fix mutex lock/unlocks symmetry in babl_fish()

In the case where the fish we are to look up is the memcpy fish we were
not locking the mutex. Whereas the common cleanup after the branches
assumes the lock is held. This is probably the cause of gimp#7632.

tools: include more conversions in babl-lut-verify

tools: use _putenv_s() instead of setenv() on Windows.

setenv() does not exist on Windows API, hence babl tools fail to build.
Fixing the issue by using an alternative Windows API.

tools: rewrite babl-lut-verify to be more generic

tools: include CMYK encodings in babl-benchmark

babl: add 4:8 luts

tools/babl-benchmark: print LUT/REF markers

babl: do not load cache if BABL_INHIBIT_CACHE is set in environment

tools: inhibit cache use in lut-verify and babl-benchmark

sse2-float: correctly handle associted alpha in remainder

babl: fix fix bpp initialization when loaded from cache

babl: clean up compiler warnings

babl: do luts for 2->16 byte fishes

babl: initialize bpp after reading cost in fish cache

babl: make unified alpha conversion branchless

babl: improve LUT debug info

babl: add support for 2,2 LUTs

babl: only show active luts during gc on info_level>=4

new symbol babl_fish_get_process

This permits some APIs to avoid a function call per dispatch,
specifically the samplers in GeglBuffer.

babl: remove optional instrumentation

babl: add LUT debugging configurable through BABL_LUT_INFO environment variable

It is a number ranging from 0 to 5 if with more information provided at
higher levels.

babl: measure performance of LUTs and only replace if faster

With this also enabling replacement of conversions where spaces do not
differ.

babl: LUTs for 16byte destination encodings

babl: do LUTs for 1 and 2 byte encodings

tools: add RGB variants to babl-benchmark

babl: also do auto LUTs for 3-3 bpp and 3-4bpp paths

babl: lut remove unused u16 code

tools: use rand() as fallback of random().

This fixes building babl on Windows as the random() API is not available
there (unlike rand()). Windows docs says it's not cryptographically
secure but for this tool usage, it's probably enough and I see we use
the same function replacement tricks on other tool sources.

Revert "make trampoline for lut processing"

This reverts commit b3e884edf3b5c58fb4c2cede1346bd8a9d9c4a1e.

benchmarking on x86_64 saw no effect, and as a surprise on arm
it pessimises performance.

build: fix append of sse2 cflags

make trampoline for lut processing

build: append list of one element rather than string

tools: add babl-lut-verify

ctx: factor lut processing out of path_process

babl: LUTs fix typo in timelimit check

babl: free LUTs that have been unused for more than 5 minutes

babl: lut, tidy u8 code - disable u16/half variant

babl: avoid using half initialized LUTs when used from threads

babl: add sse2 flags when building RGB converter

babl: LUT cleanups

babl: use LUT for 16bit to 8bit conversions

babl: remove some now dead u8 code

babl: add lut for some u8 conversions

The fish path now marks created fishes that shall be monitored for
usage and have full 24bit LUTs generated on demand when limits are
exceeded. This 3-4x increases the throughput.

tools: rework babl-benchmark to operate on sets resembling workingspace + GEGL + display

babl: add u8 code paths to trc/matrix space conversion

This is relevant for ARM, on modern x86_64 at least it is cheaper to
convert to/from float with external loops.

babl: adjust default path parameters to a path_length of 3

Also adjust so that the max additional bits of depth is 2, this makes us
catch some fast paths that are missed by very promising slow two-step
converters.

babl: remove allocations in u8 variants of RGB space conversions

babl: cpuaccel fix arm neon detect

babl: cpuaccel fix arm neon detect

babl: add more missing SIMD build related files

babl: add missing files babl-trc.[ch]

extensions/cairo: add a correct enough for 8bit cairo32 to cairo24 conversion

extensions: reduce amount of competing conversions

Avoid loading extensions if a variant with better SIMD support
is expected to exist. We know all variants that are expectd to
exist and thus can register only the fastest known variant.

This avoids combinatorial explosion of conversions to test.

tools: babl-verify make verify do space free conversions by default, it is more useful

extensions: cairo remove a broken conversion, fix RGBA u8 to cairo reference

multipass build, with SIMD variants for x86_64 and arm

Some extensions have their own separate build, so does the
RGB color space conversions, trc computations and base
set of conversions.

babl: reduce number of pixels tested for paths to 1024, include some out of bounds pixels in start

base: x86-64-v2 and x86-64-v3 variants

build: enable -ftree-vectorize always, and -mfpu=neon on aarch64

babl: sync cpuaccel with GEGL

gitlab-ci: disable paccache, trying to unbreak build

babl: set ecx to 0 in cpuid

This is neccesary to be able to detect AVX2 reliably.

tests: add a xyz to lab test

This is testing D50 XYZ to D50 CIE Lab the use of D50 for all XYZ and
Lab is currently implicit in all babl code.

Oklab: disable only Oklch not Oklab

Oklab itself is enabled by default now for testing, but does not seem to
produce correct values with the Oklab test vectors, even with manual
attempt at converting the test vector (presumed) CIE XYZ D65 to CIE XYZ
D50 before running the conversion.

The failure to pass symmetry tests as mentioned in issue #63 seems to be
limited to Oklch, Oklab itself is fine.

oklab: add double variant of code for model conversions

babl: adjust sanity criteria for babl_model_is_symmetric

oklab: add to meson, register model conversions

export-symbols: add babl_chromatic_adaptation_matrix

quick fix

oklab: chad back to D65

expose chromatic adaptation matrix

oklab, initial ver

gitlab-ci: remove nonexistent meson option enable-sse3

post-release version bump

release 0.1.88

babl: re-creation of palette conversions must always set data.

If we allow conversions to be recreated for palettes, it will create a
new identical conversion with collision number incremented in the name
and no user data. Yet the user data is extremely important for palette
conversions, because it contains the palette itself which is required by
the palette conversion functions. Hence we end up crashing.

Moreover, although I think that right now, only palette conversions use
the data field, it might be used for other type of data in the future.
In any case, we should always save this data along as it has been stored
there for a reason.

Finally in babl-palette code, make sure the user data is added to the
models before creating the formats and conversions as we also ended up
in crash-happy code when we didn't.

See: https://gitlab.gnome.org/GNOME/gimp/-/issues/6501

babl: do not append space to the format name when explicitly set.

When a format is named explicitly by the caller, we should just keep the
same name, not tweak it. Generating a name is only valid when no
explicit name is given. Moreover this is the behavior as described in
the docs of babl_format_new():

> If no name is provided a (long) descriptive name is used.

(which usually implies that with a name provided, this one will be used
instead of a generated name)

This is especially important for palette formats for which name is
mostly not descriptive anyway (either caller-set or generated by babl
with incremental number) and this fixes a bug when calling
babl_new_palette_with_space() with the same name and space would fail to
find the already created formats (yet would find the models and return
NULL for both formats) because it would search without the space
description appended. Yet the docs of babl_new_palette*() clearly says
as well:

> If you pass in the same name the previous formats will be provided
> again.

Moreover we actually already tweak a bit the palette format name by
appending the space pointer. No need to tweak it twice appending the
same information (space) in a different way.

See also: https://gitlab.gnome.org/GNOME/gimp/-/issues/6501

build: remove push_web target

ci/meson has started failing due to missing scp binary.

build: subproject improvements:
 - don't expose source in dependency to avoid name clashes
 - add babl path variables to dependency object
 - add babl path variables to uninstalled pc

post release version bump

release 0.1.86

NEWS: update

Another project can add babl as a subproject

ci: remove request for specific arch glibc version

We're getting the version by forcing a specific base image, and ignoring
glibc updates.

ci: use archlinux:base-20210214.0.15477 as base for building

This gives us an arch base from before the glibc version that enforces
a newer glibc package that has a hard dependency on a newer linux
kernel.

babl: babl-fish-reference remove superfluous set of dst_img->type[0]

As reported by Rafał Mikrut, fixing issue #57.

ci: Unbreak archlinux-based CI

Recently Arch has changed its glibc package to require kernel
4.4[1] as a poor man's attempt to disable faccessat2 syscall that
is known to cause problems with old libseccomp and Docker. Downgrade
glibc to the pkgrel before the change and ensure it's not upgraded.

Additionally switch to the official Arch Linux image in the library
namespace. The base image has been deprecated.

[1] https://github.com/archlinux/svntogit-packages/commit/893b1c268abc8822332655865e3d4546025a9b4b

build: revert glibc workaround for arch CI

build: add -y parameter to glibc pacman workaround

build: attempt forcing of older glibc to unbreak CI

icc: make babl_space_from_icc threadsafe

When multiple threads concurrently try to use ICC profiles / spaces
races during construction could cause broken internal representation of
profiles.

icc: add offset bounds checks to read_sign

babl: support creating babl spaces from input-class ICC profiles

post release version bump

release 0.1.84